Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add tf plans for syncing labels across Charmed HPC repositories #13

Merged

Conversation

NucciTheBoss
Copy link
Member

This PR adds a Terraform plan to our "meta" repository here that syncs our issue/pull request labels across the Charmed HPC GitHub organization. When a new PR is merged in main, GitHub Actions will run the Terraform plan to update all the labels on our main repositories. Current targeted repositories are as follows:

  • .github
  • docs
  • charmed-hpc-terraform
  • hpc-libs
  • slurm-charms
  • slurm-snap
  • slurmutils
  • filesystem-charms
  • ondemand-snap

These repositories were chosen since they are the ones we primarily work on right now, and it's not difficult to add more repositories to the plan. You can see the labels already uploaded here: https://github.com/charmed-hpc/slurmutils/labels

I also added a justfile that enables us to easily run the Terraform plans with a provided GitHub PAT. Let me know what you think!

Signed-off-by: Jason C. Nucciarone <[email protected]>
Signed-off-by: Jason C. Nucciarone <[email protected]>
Include section about how to generate PAT that can edit repository labels. Used some images since the UI can be a bit confusing.

Signed-off-by: Jason C. Nucciarone <[email protected]>
@NucciTheBoss NucciTheBoss added the feature This pull request adds a new feature to the project label Feb 5, 2025
Copy link
Contributor

@jedel1043 jedel1043 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! The README looks really nice, and the action to update the tags without having to do it from a local machine is appreciated.

@NucciTheBoss
Copy link
Member Author

LGTM! The README looks really nice, and the action to update the tags without having to do it from a local machine is appreciated.

It should work, but I might need to get a little crafty with the state management of the plan. We'll see if there needs to be a preseed - e.g. pull label data from GitHub as part of the plan - but it shouldn't be too difficult to set up if needed.

@NucciTheBoss NucciTheBoss merged commit c1da1ab into charmed-hpc:main Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature This pull request adds a new feature to the project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants